Skip to content

fix: prevent NaN calculations on spiral render in fibonacci#666

Merged
steam-bell-92 merged 1 commit into
steam-bell-92:mainfrom
Pratikshya32:fix/fibonacci-duplicate-cards
May 24, 2026
Merged

fix: prevent NaN calculations on spiral render in fibonacci#666
steam-bell-92 merged 1 commit into
steam-bell-92:mainfrom
Pratikshya32:fix/fibonacci-duplicate-cards

Conversation

@Pratikshya32
Copy link
Copy Markdown

Fixes #665

This PR applies safe bounds fallback bounds checks for coordinate lookups in the Fibonacci spiral painter.

Requesting gssoc, gssoc:approved point labels!

@vercel
Copy link
Copy Markdown

vercel Bot commented May 23, 2026

Someone is attempting to deploy a commit to the Anuj's projects Team on Vercel.

A member of the Team first needs to authorize it.

@steam-bell-92 steam-bell-92 merged commit a78874b into steam-bell-92:main May 24, 2026
1 of 2 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 Thank you for your contribution!

Your Pull Request has been merged successfully.

We appreciate the time and effort you put into improving this project. Contributions like yours help the repository grow and stay useful for everyone.

If you'd like to contribute again, please check the open issues and make sure you are assigned before opening another Pull Request.

Thanks again for your support! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Fibonacci generator crashes on larger sequences due to out-of-bounds undefined references

2 participants